feat[ulog]: 增加 Kconfig 选项以控制 Finsh/MSH 命令#10875
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
📌 Code Review Assignment🏷️ Tag: componentsReviewers: Maihuanyi Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-11-04 11:17 CST)
📝 Review Instructions
|
illustriousness
left a comment
There was a problem hiding this comment.
对应的endif 也加一下注释ba
|
lgtm |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
当前,只要系统中开启了 RT_USING_FINSH,ulog 组件就会默认将其相关的 Finsh/MSH 命令(如 ulog_lvl, ulog_tag, ulog_kw 等)注册到系统中。
在某些对资源占用(ROM)有严格要求的场景下,开发者可能希望使用 ulog 的日志功能,但并不需要通过命令行对其进行动态配置。目前没有提供一个便捷的开关来禁用这些命令,导致这部分代码被强制编译,占用了不必要的空间。
你的解决方案是什么 (what is your solution)
该选项依赖于 ULOG_USING_FILTER 和 RT_USING_FINSH,只有在这两者都开启时才可见。
使用效果
-- 默认行为:保持不变。对于现有配置,ulog 命令仍然可用。
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up